Closed
Bug 619769
Opened 14 years ago
Closed 14 years ago
FrameState::init leaks cursor when reifier.init() fails
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla6
People
(Reporter: timeless, Assigned: timeless)
References
(Blocks 1 open bug, )
Details
(Keywords: coverity, memory-leak, Whiteboard: fixed-in-cedar)
Attachments
(1 file)
|
921 bytes,
patch
|
jorendorff
:
review+
|
Details | Diff | Splinter Review |
69 FrameState::init()
70 {
89 uint8 *cursor = (uint8 *)cx->calloc(totalBytes);
90 if (!cursor)
91 return false;
92
93 #if defined JS_NUNBOX32
94 if (!reifier.init(nentries))
this leaks cursor:
95 return false;
|
||
Comment 2•14 years ago
|
||
Comment on attachment 498295 [details] [diff] [review]
reorder
Review of attachment 498295 [details] [diff] [review]:
r=me
Attachment #498295 -
Flags: review?(jorendorff) → review+
Keywords: checkin-needed
|
||
Comment 3•14 years ago
|
||
Flags: in-testsuite-
Keywords: checkin-needed
Whiteboard: fixed-in-cedar
Target Milestone: --- → mozilla6
|
||
Comment 4•14 years ago
|
||
|
|
||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Updated•7 years ago
|
Blocks: coverity-analysis
You need to log in
before you can comment on or make changes to this bug.
Description
•